Skip to content
This repository was archived by the owner on Feb 15, 2026. It is now read-only.

Comments

Coding standards and formatting#6

Merged
olibf merged 1 commit intomainfrom
feature/refactoring
Dec 17, 2025
Merged

Coding standards and formatting#6
olibf merged 1 commit intomainfrom
feature/refactoring

Conversation

@olibf
Copy link
Owner

@olibf olibf commented Dec 17, 2025

Pull Request

Description

  • refactor null checks and fix formatting inconsistencies
  • Added '.editorconfig' for consistent coding standards

Type of Change

  • Code refactoring

Changes Made

  • refactor null checks and fix formatting inconsistencies
  • Added '.editorconfig' for consistent coding standards

Testing

(no code change, only reformatting)


Note

Adds project-wide .editorconfig, replaces manual null checks with ThrowIfNull, and marks MsSql extractor helper methods as static.

  • Project config:
    • Add .editorconfig to enforce C# formatting, naming, and analyzer severities.
  • Application:
    • Replace manual null checks with ArgumentNullException.ThrowIfNull in CustomTextFormatter.Format and SchemaExportService.ExportSchemaAsync.
    • Minor logging/message formatting tweaks in SchemaExportService.
  • Infrastructure:
    • Mark MsSqlSchemaExtractor helpers ExtractTablesAsync and ExtractColumnsAsync as static.
  • CLI:
    • Minor cleanup (remove unused using; small formatting around config path validation).

Written by Cursor Bugbot for commit 6f5641a. This will update automatically on new commits. Configure here.

@olibf olibf merged commit 19c1723 into main Dec 17, 2025
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant